Skip to content
This repository was archived by the owner on Aug 19, 2022. It is now read-only.

Adding enhancer tests for better coverage#310

Merged
ianobermiller merged 2 commits intoFormidableLabs:masterfrom
AnSavvides:264-enhancer-tests
Aug 11, 2015
Merged

Adding enhancer tests for better coverage#310
ianobermiller merged 2 commits intoFormidableLabs:masterfrom
AnSavvides:264-enhancer-tests

Conversation

@AnSavvides
Copy link
Copy Markdown
Contributor

This closes #264 by bringing in better test coverage to the enhancer.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should probably be consistent and always require react/addons in tests, might help a bit with speed (just conjecture).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After some crude testing it doesn't look like it makes much of a difference (if it does, it's very marginal), but we should do this for consistency's sake anyway - so here it is: #312

@ianobermiller
Copy link
Copy Markdown
Contributor

Sweet! 🚀

ianobermiller added a commit that referenced this pull request Aug 11, 2015
Adding Enhancer tests for better coverage
@ianobermiller ianobermiller merged commit d9da13f into FormidableLabs:master Aug 11, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add tests for enhancer static prop transferring

2 participants